Skip to content

Print timing inside overall test reporting#12

Merged
vchuravy merged 1 commit into
mainfrom
mg/time
Oct 10, 2025
Merged

Print timing inside overall test reporting#12
vchuravy merged 1 commit into
mainfrom
mg/time

Conversation

@giordano

Copy link
Copy Markdown
Collaborator

Test.DefaultTestSet natively supports showing time tests took, we don't need to print it separately.

`Test.DefaultTestSet` natively supports showing time tests took, we don't need
to print it separately.
@vchuravy

Copy link
Copy Markdown
Member

Right so you are saying:

Test Summary: | Pass  Total  Time
  Overall     |    3      3  2.9s
    SUCCESS

Is sufficient

@vchuravy

Copy link
Copy Markdown
Member

Whereas before we had:

Testing finished in 2 seconds, 511 milliseconds

Test Summary: | Pass  Total  Time
  Overall     |    1      1

@vchuravy vchuravy merged commit ade40da into main Oct 10, 2025
15 checks passed
@giordano giordano deleted the mg/time branch October 10, 2025 15:43
@giordano

Copy link
Copy Markdown
Collaborator Author

Yes. I guess this part was originally written when Test.DefaultTestSet didn't support timing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants